Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

[ci skip] Replace touch method to running callbacks list #12836

Closed
wants to merge 3 commits into from
Closed

[ci skip] Replace touch method to running callbacks list #12836

wants to merge 3 commits into from

Conversation

sbagdat
Copy link
Contributor

@sbagdat sbagdat commented Nov 10, 2013

touch method was listed in Skipping Callbacks list, but it triggers after_touch callback, so I replaced it.

@sbagdat
Replace touch method to running callbacks list
658bddd 
`touch` method was listed in *Skipping Callbacks* list, but it triggers `after_touch` method, so I replaced it.
@JuanitoFatas
Copy link
Contributor

Saves the record with the updated_at/on attributes set to the current time. Please note that no validation is performed and only the after_touch callback is executed. –– from ActiveRecord::Persistence

@sbagdat
Replace touch method to running callbacks list
8994833 
`touch` method was listed in *Skipping Callbacks* list, but it triggers `after_touch` method, so I replaced it.
@sbagdat
Merge branch 'patch-1' of https://github.com/sbagdat/rails into patch-1
8c74d76
@sbagdat sbagdat closed this Nov 11, 2013
@sbagdat sbagdat deleted the patch-1 branch November 11, 2013 00:51
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

None yet
2 participants
@sbagdat @JuanitoFatas